feat: import MyBMAD web dashboard alongside VS Code extension#5
feat: import MyBMAD web dashboard alongside VS Code extension#5
Conversation
Coordinates the BMAD UI ecosystem under one repository (per BMadCode's Discord proposal) by importing the standalone Next.js dashboard from DevHDI/my-bmad@81b28ea into web/, side-by-side with the existing VS Code extension under src/. Scope is intentionally minimal: - Exact verbatim import via `git archive HEAD` from DevHDI/my-bmad@81b28ea (the relicense-to-MIT commit on main) - Zero changes inside src/ — the extension build, lint, type-check, and tests are unaffected - Root eslint.config.mjs ignores web/** so the extension's strict TypeScript rules do not lint the Next.js code with extension-only policies - Root .prettierignore ignores web/ for the same reason - Root README updated with a "Repository structure" section documenting the side-by-side layout Out of scope (deferred to follow-up PRs once Brian and Evie weigh in): - pnpm workspace / monorepo restructuring - Shared parser package (each subproject keeps its own parser for now) - CI for the web dashboard - Preview deployment / hosting Note on web/.github/: the import is verbatim, so a nested .github/ folder is included for provenance. GitHub only executes workflows under the repository's top-level .github/workflows, so the nested ones do not run. Verification: - pnpm install --frozen-lockfile (root): clean - pnpm typecheck (extension + webview): clean - pnpm test (extension): 706/706 - pnpm build (extension): produces extension + webview bundles - pnpm lint (extension): clean (only the 2 pre-existing no-console warnings; no web/ files scanned) - cd web && pnpm install --frozen-lockfile: clean - cd web && pnpm tsc --noEmit: clean - cd web && pnpm test: 151/151 - cd web && pnpm build: clean
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Coordinates the BMAD UI ecosystem under one repository (per @bmadcode's Discord proposal) by importing the standalone Next.js dashboard from
DevHDI/my-bmad@81b28eaintoweb/, side-by-side with the existing VS Code extension undersrc/.This PR is intentionally a draft — it asks for approval on the structure before any merge. No
src/file is touched; @evie's extension is unchanged.Scope (intentionally minimal)
git archive HEADfromDevHDI/my-bmad@81b28ea(the relicense-to-MIT commit onmain)src/— extension build, lint, type-check, tests are unaffectedeslint.config.mjsignoresweb/**so the extension's strict TypeScript rules don't lint the Next.js code with extension-only policies.prettierignoreignoresweb/for the same reasonREADME.mdupdated with a Repository structure section documenting the side-by-side layoutOut of scope (deferred to follow-up PRs)
parserpackage extracted from both subprojects (the two implementations cover similar ground but have very different runtimes — VS Code extension vs. multi-user Next.js)Note on
web/.github/The import is verbatim, so a nested
.github/folder is included for provenance. GitHub only executes workflows under the repository's top-level.github/workflows, so the nested ones atweb/.github/workflows/are imported as files but do not run as active GitHub Actions.License
DevHDI/my-bmadwas relicensed from AGPL-3.0 to MIT inDevHDI/my-bmad@81b28eaprior to this import, so the entire repository is now MIT-consistent. The importedweb/LICENSEcarries the standard MIT text.Verification
Both subprojects pass all checks independently:
Extension (root) — unchanged:
pnpm install --frozen-lockfile— cleanpnpm typecheck(extension + webview) — cleanpnpm test— 706/706pnpm build— produces extension + webview bundlespnpm lint— clean (only the 2 pre-existingno-consolewarnings; noweb/files scanned)Web (
web/) — newly imported:pnpm install --frozen-lockfile— cleanpnpm tsc --noEmit— cleanpnpm test— 151/151pnpm build— cleanOpen questions for @bmadcode and @evie
web/(proposed),dashboard/,apps/web/?pnpm-workspace.yamland anapps/+packages/layout? Doing it now would be a single big move; doing it later as its own PR keeps this one focused on the import.parserpackage from bothsrc/extension/parsers/andweb/src/lib/bmad/makes sense long-term, but it's a non-trivial refactor that touches both surfaces. Worth a dedicated discussion before any code change.DevHDI/my-bmad: archive after this lands, or keep it as a public mirror?Note on review
216 files changed — most of them are an exact import. The history of those files lives at
DevHDI/my-bmadand includes 5 already-reviewed-and-merged PRs. Please focus the review on:README.md,eslint.config.mjs,.prettierignore, the newweb/directory at the top)